Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CheckoutPage: submit button was enabled prematurely for onetime payments #111

Merged
merged 3 commits into from
Sep 29, 2021

Conversation

Gnito
Copy link
Contributor

@Gnito Gnito commented Sep 28, 2021

  1. If default payment method and onetime payment method were toggled, submit button was enabled prematurely. There was no check for valid Stripe Element card value, when selecting onetime payment option.
  2. Reselecting saved-payment-method flow after receiving an error from initial attempt of onetime-payment, didn't send payment method to Stripe correctly.

@Gnito Gnito force-pushed the fix-disable-checkoutpage-submit-button branch 3 times, most recently from a9671c8 to 0316f7c Compare September 28, 2021 13:59
Copy link
Contributor

@kpuputti kpuputti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small naming suggestions, otherwise looks ok to me. Quite a beast, this form! I wonder if it should be split down and refactored at some point?

src/containers/CheckoutPage/CheckoutPage.duck.js Outdated Show resolved Hide resolved
src/containers/CheckoutPage/CheckoutPage.js Outdated Show resolved Hide resolved
@Gnito Gnito force-pushed the fix-disable-checkoutpage-submit-button branch from 0316f7c to 0c7b22c Compare September 29, 2021 15:43
@Gnito Gnito merged commit 04f1d95 into main Sep 29, 2021
@Gnito Gnito deleted the fix-disable-checkoutpage-submit-button branch September 29, 2021 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants